Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CHANGES.md for 2.6 release #34

Merged
merged 23 commits into from
Feb 14, 2023
Merged

Update CHANGES.md for 2.6 release #34

merged 23 commits into from
Feb 14, 2023

Conversation

richfromm
Copy link
Owner

@richfromm richfromm commented Feb 14, 2023

  • During parsing, skip any attached files that have been deleted
  • Add --ignore-file-not-found option to ignore any attached files that can't
    be found when downloading files
    • The default is to raise an exception
    • This can occur if a file is deleted from Slack after the export, and
      before running this script to perfom the Discord import
  • Allow for re-use of downloaded files
    • Previously, even if you specified an existing directory with
      --downloads-dir, the file would always be freshly downloaded.
    • Now, if using the option, the existing file will be re-used, as long as
      a file exists with the same name, and the file size matches the
      Content-Length HTTP response header.
    • Also wrap the downloads in a progress bar with tqdm

Also:

not sizing yet, which I suspect might be a problem
I don't have a good way of truly testing this locally
and switch paypal to svg
but the list item itself is still markdown
these are for the donation buttons
I'm doubtful this will make a difference
and use <li> for list item
instead of CSS
which github md doesn't seem to actually support:
https://github.com/orgs/community/discussions/22728
keep image link as HTML for now
with just HTML for the img tag
stick with the mostly markdown (plus HTML IMG) that works
just experiment now with height 40 vs. 32
finalize (I hope) the markdown
remove unused image files
using svg now (not png)
only paypal is local, github comes from elsewhere
README.md Outdated
contribution, you can make a donation via either:

* [GitHub Sponsors](https://github.com/sponsors/richfromm)[<img src="https://github.githubassets.com/pinned-octocat.svg" width="auto" height="32" valign="middle" hspace="10">](https://github.com/sponsors/richfromm)
* [PayPal](https://www.paypal.com/donate/?business=J46TL293CQ2M2&no_recurring=0&currency_code=USD)[<img src="paypal.svg" width="auto" height="32" valign="middle" hspace="10">](https://www.paypal.com/donate/?business=J46TL293CQ2M2&no_recurring=0&currency_code=USD)
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was shocklingly hard to get right. See all of the trial and error commits.

Typically I've been merging PR's, but I'm going to squash this one.

@@ -2,6 +2,22 @@

## Current releases, on [this fork](https://github.com/richfromm/slack2discord)

### 2.6
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will tag 2.6 after this merges

@richfromm richfromm merged commit ed92448 into master Feb 14, 2023
@richfromm richfromm deleted the donations branch February 14, 2023 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant